Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[UI] Fix Visibility of Dark/Light Mode Toggle and Signup Button in Mobile View #206

Merged
merged 2 commits into from
Jul 30, 2024

Conversation

akshansh-modi
Copy link
Contributor

@akshansh-modi akshansh-modi commented Jul 4, 2024

Notes for Reviewers

This PR fixes #203
It addresses the issue where the dark/light mode toggle and signup button were not visible in mobile view. The display property for these elements was set to none in mobile view. This PR ensures that both the dark/light mode toggle and signup button are now visible in both desktop and mobile views.

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Akshansh <[email protected]>
Copy link

netlify bot commented Jul 4, 2024

Deploy Preview for meshery-play ready!

Name Link
🔨 Latest commit b61712c
🔍 Latest deploy log https://app.netlify.com/sites/meshery-play/deploys/66a93b397d582c0008e825b8
😎 Deploy Preview https://deploy-preview-206--meshery-play.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@akshansh-modi
Copy link
Contributor Author

@Ashparshp please accept the pull request so it can get merged, or there are some changes that I need to make?

Copy link

@Ashparshp Ashparshp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Ashparshp Ashparshp removed the request for review from captain-Akshay July 6, 2024 21:57
@akshansh-modi
Copy link
Contributor Author

akshansh-modi commented Jul 8, 2024

Hi @Ashparshp, I noticed that the PR is still marked as 'review required' even though you've approved it. Could you please confirm if everything has been completed on your end for the review process? Let me know if there's anything else needed to move it forward. Thanks!

Copy link
Contributor

@Yashsharma1911 Yashsharma1911 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Yashsharma1911 Yashsharma1911 merged commit 9005b22 into meshery:master Jul 30, 2024
2 checks passed
@Yashsharma1911
Copy link
Contributor

Thankyou @akshansh-modi for fixing this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dark/Light Mode and Signup Button Not Visible in Mobile View
3 participants